feat(editor): tint tab bar with active connection color#333
Merged
Conversation
Contributor
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (2 files)
Previous Review Summaries (3 snapshots, latest commit 0614881)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit 0614881)Status: No Issues Found | Recommendation: Merge Files Reviewed (10 files)
Previous review (commit a463e7b)Status: No Issues Found | Recommendation: Merge Files Reviewed (10 files)
Previous review (commit 80c6da7)Status: No Issues Found | Recommendation: Merge Files Reviewed (1 file)
Reviewed by kimi-k2.7-code-20260612 · Input: 127.9K · Output: 13.6K · Cached: 611.4K |
Extend the connection-color tab bar so every accent follows the active connection, falling back to the default blue when none is active: - Active-tab indicator line uses the connection color with a soft glow - Active tab carries an accent-tinted body gradient over the content bg - Inactive tabs pick up an accent wash on hover instead of a flat grey - Loading bar and rename input border follow the connection color - Tab bar uses a vertical accent gradient with an accent-tinted bottom border so the strip reads as part of the connection - Scroll arrows and new-tab buttons use theme-safe hover colors
Collaborator
|
I really love this feature — tinting the workspace with the active connection's color makes it so much easier to tell environments apart at a glance. I pushed a few additions that carry the accent through the whole tab bar:
What do you think? The accent intensity is just a couple of hex alpha values, so I'm happy to dial it up or down if it feels too strong or too subtle. |
Contributor
Author
|
Thanks so much — I really like all the integrations you added. I pushed a small follow-up that just tones the accent down a bit — I personally lean toward a slightly less vibrant look:
Let me know what you think |
Collaborator
|
LGTM. If @NewtTheWolf doesn't have anything to add, I'd go ahead and merge it. |
Split-pane panel headers and the split-view connection switcher now use each connection's accent color (getConnectionAccent) instead of a fixed blue, matching the tinted editor tab bar. Inactive items reuse the same accent at a dimmer alpha.
Collaborator
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #330
Applies the connection's accent color as a subtle tint on the editor tab bar, for at-a-glance connection identity.